Node / meshtastic / Meshtastic-Android / files / specs / 20260520-153428-remove-admin-channel-toggle / research.md

Displaying Rendered • View raw • Download

This feature has no unresolved unknowns. The technical context is fully specified by the feature spec and existing codebase analysis.

• Decision: Remove lines 207–214 in T383838SecurityConfigScreen.kt (HorizontalDivider + SwitchPreference block)

• Rationale: Confirmed via code inspection — these are the only lines rendering the T383838admin_channel_enabled toggle

• Alternatives considered: None — this is the only location

• Decision: Remove line 49 (T383838import org.meshtastic.core.resources.legacy_admin_channel)

• Rationale: After removing the SwitchPreference that uses T383838Res.string.legacy_admin_channel, this import becomes unused. Leaving it would fail lint checks.

• Alternatives considered: Leave import (rejected — violates Zero Lint Tolerance principle)

• Decision: Do NOT modify T383838admin_channel_enabled proto field

• Rationale: Per spec non-goals and Constitution §IV (Privacy First / read-only proto submodule). The field remains for firmware communication and backward compatibility.

• Alternatives considered: Full removal including proto (rejected — requires upstream proto change, out of scope)

• Decision: Leave T383838legacy_admin_channel string resources intact

• Rationale: String resources are Crowdin-managed. Removal is a separate cleanup task. No lint violation from unused string resources.

• Alternatives considered: Remove strings (rejected — Crowdin-managed, separate concern)

• Decision: Run T383838./gradlew :feature:settings:allTests :feature:settings:compileKotlinJvm

• Rationale: Covers all KMP targets for the settings module. T383838compileKotlinJvm ensures desktop target compiles correctly after the removal.

• Alternatives considered: Full T383838assembleDebug (not needed — change is isolated to one module)

All technical context was resolved from spec + codebase inspection. No external research required.